Skip to content

Remove obsolete modern xattrs metadata backend - #923

Open
asaah262 wants to merge 3 commits into
opencloud-eu:mainfrom
asaah262:fix/3048-deprecate-and-remove-xattrs-metadata
Open

asaah262 wants to merge 3 commits into
opencloud-eu:mainfrom
asaah262:fix/3048-deprecate-and-remove-xattrs-metadata

Conversation

@asaah262

Copy link
Copy Markdown

Summary

Closes opencloud-eu/opencloud#3048

Remove the obsolete modern XattrsBackend implementation from Reva.

Changes

  • Remove xattrs metadata backend selection from POSIX. Hybrid remains the default.
  • Remove xattrs metadata backend selection from DecomposedFS and DecomposedS3. MessagePack remains the default.
  • Delete pkg/storage/pkg/decomposedfs/metadata/xattrs_backend.go.
  • Remove obsolete xattrs references from the corresponding test helpers.
  • Leave the legacy implementation under pkg/storage/utils/decomposedfs untouched.

Testing

  • Added regression tests verifying that POSIX and DecomposedFS reject the removed backend.
  • Verified that the default metadata backends remain unchanged.
  • go build ./... passed.
  • Relevant Reva tests and vet checks passed.

Compatibility

OpenCloud does not expose configuration for selecting the obsolete xattrs metadata backend.

Removing the exported NewXattrsBackend() function requires a corresponding change in OpenCloud's decomposedfs CLI. That change has been implemented and tested separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

deprecate and remove xattrs metadata backend

1 participant